-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use i18n Keys #5286
Use i18n Keys #5286
Conversation
Feel free to do so. I think I'll be able to resolve the conflicts myself later this week (apparently I should be able to resolve them 😅) |
@amanape , I think OpenHands was able to clean this up, please TAL! |
@amanape any chance you have a few minutes to review this PR please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay, LGTM!
@Vaishakh-SM just needs a final conflict resolution for merge if you have a few minutes 🙏 |
End-user friendly description of the problem this fixes or functionality that this introduces
Right now, several instances of plain English language are used in the codebase. Using i18 keys would ensure better inclusivity by enabling users to use different languages.
Link of any specific issues this addresses
Issue#4280